Skip to content

ImperialLinkedIn#123

Merged
fpseverino merged 10 commits into
vapor-community:mainfrom
mynona:main
Oct 10, 2025
Merged

ImperialLinkedIn#123
fpseverino merged 10 commits into
vapor-community:mainfrom
mynona:main

Conversation

@mynona

@mynona mynona commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

Add ImperialLinkedIn

  • Follows the same pattern as ImperialGoogle standard
  • Utilizes the new linkedin API
  • Added detailed integration notes
  • but I am unsure about the tests as I could not run any test of any Imperial package locally with success. I followed Severins proposal to copy paste ImgurRoute

@fpseverino

@codecov

codecov Bot commented Oct 10, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.98%. Comparing base (19348b9) to head (1e9308e).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Sources/ImperialLinkedIn/LinkedInAuth.swift 83.33% 2 Missing ⚠️
...ources/ImperialLinkedIn/LinkedInCallbackBody.swift 0.00% 1 Missing ⚠️
Sources/ImperialLinkedIn/LinkedInRouter.swift 97.05% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
+ Coverage   82.41%   82.98%   +0.56%     
==========================================
  Files          59       63       +4     
  Lines         802      852      +50     
==========================================
+ Hits          661      707      +46     
- Misses        141      145       +4     
Files with missing lines Coverage Δ
Sources/ImperialLinkedIn/LinkedIn.swift 100.00% <100.00%> (ø)
...ources/ImperialLinkedIn/LinkedInCallbackBody.swift 0.00% <0.00%> (ø)
Sources/ImperialLinkedIn/LinkedInRouter.swift 97.05% <97.05%> (ø)
Sources/ImperialLinkedIn/LinkedInAuth.swift 83.33% <83.33%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fpseverino fpseverino left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, thank you very much! Just a few minor oversights to point out

Comment thread Sources/ImperialLinkedIn/Documentation.docc/Documentation.md Outdated
Comment thread Sources/ImperialLinkedIn/Documentation.docc/Documentation.md Outdated
Comment thread Sources/ImperialLinkedIn/Documentation.docc/Documentation.md Outdated
Comment on lines +40 to +41
print("====")
print(url.absoluteString)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These prints may be left over from debugging, please remove them

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thank you for pointing it out. Oversaw it and will fix it.

mynona and others added 4 commits October 10, 2025 23:33
Co-authored-by: Francesco Paolo Severino <96546612+fpseverino@users.noreply.github.com>
Co-authored-by: Francesco Paolo Severino <96546612+fpseverino@users.noreply.github.com>
Co-authored-by: Francesco Paolo Severino <96546612+fpseverino@users.noreply.github.com>
@mynona mynona requested a review from fpseverino October 10, 2025 21:48
@mynona

mynona commented Oct 10, 2025

Copy link
Copy Markdown
Contributor Author

@fpseverino Hi, didn't want to request a review instantly. Was a mistake. I apologize.

@fpseverino

Copy link
Copy Markdown
Member

@fpseverino Hi, didn't want to request a review instantly. Was a mistake. I apologize.

No worries 😉👍

@fpseverino fpseverino left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@fpseverino fpseverino merged commit e1ecc85 into vapor-community:main Oct 10, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants